From 26938b0452b6c7371d3466fccefb85afe40172b3 Mon Sep 17 00:00:00 2001
From: tsteven4 <13596209+tsteven4@users.noreply.github.com>
Date: Thu, 25 May 2023 06:06:37 -0600
Subject: [PATCH] switch some builds to Qt 6.5.1 (#1119)

* update some builds to Qt 6.5.1

* conditional code for QVersionNumber evolution
---
 .github/workflows/macos.yml   |  2 +-
 .github/workflows/windows.yml |  2 +-
 gui/upgrade.cc                | 49 ++++++++++++++++++++---------------
 tools/ci_install_qt.sh        |  1 +
 4 files changed, 31 insertions(+), 23 deletions(-)

diff --git a/.github/workflows/macos.yml b/.github/workflows/macos.yml
index ba7eee630..39bd2881c 100644
--- a/.github/workflows/macos.yml
+++ b/.github/workflows/macos.yml
@@ -33,7 +33,7 @@ jobs:
             GENERATOR: 'Ninja'
             RELEASE: true
             os: macos-12
-          - QT_VERSION: '6.5.0'
+          - QT_VERSION: '6.5.1'
             XCODE_VERSION: '14.2'
             GENERATOR: 'Ninja'
             RELEASE: false
diff --git a/.github/workflows/windows.yml b/.github/workflows/windows.yml
index 3eb338be2..b7519ce66 100644
--- a/.github/workflows/windows.yml
+++ b/.github/workflows/windows.yml
@@ -60,7 +60,7 @@ jobs:
             GENERATOR: 'Visual Studio 17 2022'
             RELEASE: false
             os: windows-latest
-          - QT_VERSION: '6.2.4'
+          - QT_VERSION: '6.5.1'
             ARCH: 'amd64'
             HOST_ARCH: 'amd64'
             COMPILER: 'msvc2019_64'
diff --git a/gui/upgrade.cc b/gui/upgrade.cc
index b17f67270..73211c4cf 100644
--- a/gui/upgrade.cc
+++ b/gui/upgrade.cc
@@ -20,27 +20,29 @@
  */
 
 #include "upgrade.h"
-#include <qglobal.h>                        // for qDebug
-#include <QByteArray>                       // for QByteArray
-#include <QDebug>                           // for QDebug
-#include <QLocale>                          // for QLocale
-#include <QSysInfo>                         // for QSysInfo
-#include <QUrl>                             // for QUrl
-#include <QVariant>                         // for QVariant
-#include <QVersionNumber>                   // for QVersionNumber, operator<, operator==
-#include <Qt>                               // for ISODate, RichText
-#include <QDesktopServices>                 // for QDesktopServices
-#include <QNetworkAccessManager>            // for QNetworkAccessManager
-#include <QNetworkReply>                    // for QNetworkReply, QNetworkReply::NoError
-#include <QNetworkRequest>                  // for QNetworkRequest, QNetworkRequest::ContentTypeHeader, QNetworkRequest::HttpReasonPhraseAttribute, QNetworkRequest::HttpStatusCodeAttribute, QNetworkRequest::NoLessSafeRedirectPolicy, QNetworkRequest::RedirectPolicyAttribute, QNetworkRequest::Redi...
-#include <QMessageBox>                      // for QMessageBox, QMessageBox::Yes, operator|, QMessageBox::No
-#include <QtXml/QDomDocument>               // for QDomDocument
-#include <QtXml/QDomElement>                // for QDomElement
-#include <QtXml/QDomNode>                   // for QDomNode
-#include <QtXml/QDomNodeList>               // for QDomNodeList
-#include "gbversion.h"                      // for VERSION
-#include "babeldata.h"                      // for BabelData
-#include "format.h"                         // for Format
+
+#include <QByteArray>             // for QByteArray
+#include <QDebug>                 // for QDebug
+#include <QDesktopServices>       // for QDesktopServices
+#include <QDomDocument>           // for QDomDocument
+#include <QDomElement>            // for QDomElement
+#include <QDomNode>               // for QDomNode
+#include <QDomNodeList>           // for QDomNodeList
+#include <QLocale>                // for QLocale
+#include <QMessageBox>            // for QMessageBox, QMessageBox::Yes, operator|, QMessageBox::No
+#include <QNetworkAccessManager>  // for QNetworkAccessManager
+#include <QNetworkReply>          // for QNetworkReply, QNetworkReply::NoError
+#include <QNetworkRequest>        // for QNetworkRequest, QNetworkRequest::ContentTypeHeader, QNetworkRequest::HttpReasonPhraseAttribute, QNetworkRequest::HttpStatusCodeAttribute, QNetworkRequest::NoLessSafeRedirectPolicy, QNetworkRequest::RedirectPolicyAttribute, QNetworkRequest::RedirectionTar...
+#include <QSysInfo>               // for QSysInfo
+#include <QUrl>                   // for QUrl
+#include <QVariant>               // for QVariant, operator!=
+#include <QVersionNumber>         // for QVersionNumber, operator<, operator==
+#include <Qt>                     // for ISODate, RichText
+#include <QtGlobal>               // for qDebug, qsizetype, QT_VERSION, QT_VERSION_CHECK
+
+#include "babeldata.h"            // for BabelData
+#include "format.h"               // for Format
+#include "gbversion.h"            // for VERSION
 
 
 #if 0
@@ -170,8 +172,13 @@ UpgradeCheck::updateStatus UpgradeCheck::getStatus()
 // this function to sort that out. (See what I did there? Bwaaaahah!)
 bool UpgradeCheck::suggestUpgrade(const QString& from, const QString& to)
 {
+#if (QT_VERSION < QT_VERSION_CHECK(6, 4, 0))
   int fromIndex = 0;
   int toIndex = 0;
+#else
+  qsizetype fromIndex = 0;
+  qsizetype toIndex = 0;
+#endif
   QVersionNumber fromVersion  = QVersionNumber::fromString(from, &fromIndex);
   QVersionNumber toVersion  = QVersionNumber::fromString(to, &toIndex);
 
diff --git a/tools/ci_install_qt.sh b/tools/ci_install_qt.sh
index f56dfc251..611aeb291 100755
--- a/tools/ci_install_qt.sh
+++ b/tools/ci_install_qt.sh
@@ -22,6 +22,7 @@ qtdatavis3d \
 qtlottie \
 qtnetworkauth \
 qtquick3d \
+qtquick3dphysics \
 qtquicktimeline \
 qtwebglplugin \
 qtshadertools \
-- 
2.30.2